Skip to content

Conversation

@hadley
Copy link
Member

@hadley hadley commented Aug 4, 2025

Initial prototype by claude, but lots of iteration by me.

Fixes #1466

@hadley hadley requested a review from gaborcsardi August 5, 2025 16:34
@hadley
Copy link
Member Author

hadley commented Aug 5, 2025

@gaborcsardi would you mind taking a look at this one? It'd be useful if you'd try running it on a couple of projects because it's hard to test the output.

@gaborcsardi
Copy link
Member

I like it!

── Slowest tests: ───────────────────────────────────────────────────────────────────────────────────────
[4.19s] test-watcher.R: watcher works correctly
[1.73s] test-parallel-crash.R: crash
[1.17s] test-parallel-errors.R: error in parallel setup code
[0.86s] test-parallel.R: snapshots
[0.83s] test-parallel.R: snapshots are removed if test file has no snapshots
[0.82s] test-parallel.R: new snapshots are added
[0.76s] test-parallel.R: ok
[0.75s] test-parallel.R: snapshots are removed if test file is removed
[0.73s] test-parallel-outside.R: error outside of test_that()
[0.68s] test-parallel.R: fail
... and 482 more slow tests

I understand where this is coming from, but to see that that I have 482 (!) more slow tests is kind of demoralizing....

@hadley hadley merged commit fa29578 into main Aug 6, 2025
11 checks passed
@hadley hadley deleted the slow-reporter branch August 6, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reporter for finding slow tests

3 participants